Skip to content

re-add 64bits tests (skipped on 32bits) #320

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

remicollet
Copy link
Collaborator

I also add the skipif.inc which was obviously missing in the archvie.

@@ -74,6 +74,7 @@ Fixes
<file role='src' name='fastlz/fastlz.c'/>
<file role='src' name='fastlz/fastlz.h'/>
<dir name="tests">
<file role='test' name='skipif.inc'/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@sodabrew
Copy link
Contributor

sodabrew commented Feb 9, 2017

Thanks very much for moving quickly to address my concern! I put together a smaller test set in #320 and want to check in with you before I move on one or the other of the PRs.

@sodabrew sodabrew modified the milestone: 3.0.2 Feb 9, 2017
@sodabrew sodabrew mentioned this pull request Feb 9, 2017
@sodabrew
Copy link
Contributor

Thank you! Using smaller #321.

@sodabrew sodabrew closed this Feb 10, 2017
@remicollet remicollet deleted the issue-tests branch February 10, 2017 14:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants